Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add marbl #493

Merged
merged 4 commits into from
Aug 8, 2024
Merged

Add marbl #493

merged 4 commits into from
Aug 8, 2024

Conversation

mnlevy1981
Copy link
Contributor

Description of changes

Update CESM component environment variables when turning MARBL tracers on changes expected default values.
Also removed default values that are based on having POP2 as the ocean component since it is no longer available.

Specific notes

Want CCSM_BGC=CO2A and CCSM_CO2_PPMV=284.317 when MOM6 is configured with MARBL tracers.

Contributors other than yourself, if any:

CMEPS Issues Fixed (include github issue #):

Are changes expected to change answers? This is technically bit-for-bit since it will not change answers unless MARBL is turned on and there are currently no tests with MARBL active.

Any User Interface Changes (namelist or namelist defaults changes)?

Testing performed

I've run aux_mom with these changes, and confirmed bfb in non-MARBL configurations.

@mnlevy1981
Copy link
Contributor Author

We can't make cesm3_0_alpha03b without this PR merged in, but it can go into alpha03a (it doesn't require any specific tags in alpha03b)

@mnlevy1981
Copy link
Contributor Author

With the change to the plans database, this is needed in 3_0_alpha03a (the MOM6 tag with MARBL is coming in that tag, not alpha03b)

Copy link
Collaborator

@jedwards4b jedwards4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jedwards4b jedwards4b merged commit d96fdf7 into ESCOMP:main Aug 8, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants